Skip to content

Conversation

catrinjo
Copy link
Member

@catrinjo catrinjo commented Oct 15, 2026 •

This PR was closed and merged by accident. Another PR for Update MainPage Display Sections will follow shortly with the correct and final changes.
.
.
This PR adds:

  • Optional rank parameter to PassiveGameCard component.
  • Top 3 Games section
  • Top 10 Games by Genre section
  • Discounts by Genre section
  • Recent reviews section
  • Searchable genre dropdowns for filtering
  • Scaleability of mainpage and card components
  • Tests for mainpage

Note: Reviews and ratings for games are planned to be implemented in a future issue, as such this implementation is not displayed on the mainpage for this delivery.

catjoha04-byte added 3 commits October 14, 2026 10:32
add optional rank badge parameter to passivegamecard, allowing for numbered rank visualization for games when enabled
update mainpage by adding more sections and visuals, showing a Top 3 Games section, and sections based on genre selection that display top 10 games or discounted games of that genre. Additionally included a hard-coded recent reviews section.
@catrinjo catrinjo self-assigned this Oct 15, 2026
@catrinjo catrinjo requested a review from a team as a code owner October 15, 2026 21:34
@catrinjo catrinjo linked an issue Oct 15, 2026 that may be closed by this pull request
8 tasks
@catrinjo catrinjo marked this pull request as draft October 15, 2026 21:35
@catrinjo catrinjo marked this pull request as ready for review October 16, 2026 06:19
@catrinjo catrinjo marked this pull request as draft October 16, 2026 06:22
@catrinjo catrinjo marked this pull request as ready for review October 16, 2026 06:56
…nto 39-update-main-page-display-sections
Copy link
Member

@elinejaa elinejaa left a comment •

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally looks really good, great layout! Just a few changes I would like to see:

  • Fade on passive game title should only appear if full title cant be shown
  • Error message on no discounted game in genre should specify that there are no discounted games, not "no games"
  • Entire page spread should be wider, there is a lot of empty space on the page
  • Top 10 Games by Genre and Discounts by Genre filters should be stored in session storage

@elinejaa elinejaa closed this pull request by merging all changes into main in a3f89ca Oct 16, 2026
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Main page with more display sections
2 participants